[Bug Fix] Update "partition-spec" and "schema" for V1 table in update_table_metadata#547
[Bug Fix] Update "partition-spec" and "schema" for V1 table in update_table_metadata#547HonahX wants to merge 1 commit intoapache:mainfrom
Conversation
Fokko
left a comment
There was a problem hiding this comment.
I agree that it is more correct, however they are officially deprecated (for quite a while). WDYT @rdblue @danielcweeks?
Let's wait for what their thoughts are. Another route would be to do this in Pydantic when serializing to JSON.
|
This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that's incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@iceberg.apache.org list. Thank you for your contributions. |
|
This pull request has been closed due to lack of activity. This is not a judgement on the merit of the PR in any way. It is just a way of keeping the PR queue manageable. If you think that is incorrect, or the pull request requires review, you can revive the PR at any time. |
"partition-spec" and "schema" are deprecated but required field in v1 table metadata. For backward compatibility, I think we should to update "partition-spec" when applying
SetDefaultSpecUpdateand "schema" when applyingSetCurrentSchemaUpdate.